Skip to content

feat(cost-model): migration of compute cost parts #32

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
lanlou1554 wants to merge 52 commits into from
Closed

feat(cost-model): migration of compute cost parts #32

lanlou1554 wants to merge 52 commits into from

Conversation

@lanlou1554
Copy link
Contributor

@lanlou1554 lanlou1554 commented Nov 14, 2024

No description provided.

@lanlou1554 lanlou1554 force-pushed the cost-model-migration branch 3 times, most recently from 4090d0e to 9eed1c4 Compare November 15, 2024 17:51
lanlou1554 and others added 20 commits November 16, 2024 11:25
@lanlou1554
Partial migration of filter
c92a3af
@lanlou1554
Change col to attr
7b0158c
@xx01cyx @lanlou1554
implement cost computation for limit
81f8d50
@xx01cyx @lanlou1554
add author
2a5740e
@xx01cyx @lanlou1554
introduce ColumnCombValueStats
f7f6857
@xx01cyx @lanlou1554
refactor AttributeCombValueStats and introduce statistic-related data…
be430ac 
… structures
@lanlou1554
Change col to attr in filter
089cfef
@lanlou1554
Complete partial implementation of filter
69607f1
@lanlou1554
Add get_attribute_comb_stats
6518e00
@lanlou1554
Finish first draft version of filter functionality
59a8889
@lanlou1554
Add comment for the guideline of re-designing PredicateNode
5070a78
@xx01cyx @lanlou1554
introduce IdPred and make AttributeRefPred store table id and attr index
740ab11
@xx01cyx @lanlou1554
add get method for id pred and add comments
85cd0d1
@xx01cyx @lanlou1554
add check for derived column in AttributeRefPred
7775b88
@xx01cyx @lanlou1554
make get_attributes_comb_statistics return Option
b60c632
@xx01cyx @lanlou1554
implement agg cost computation
3646eca
@xx01cyx @lanlou1554
move filter-related constants to stats crate
db555ff
@xx01cyx @lanlou1554
fix clippy
64f4a10
@lanlou1554
Resolve the optional comb stats, remove table id in filter
cafd01c
@lanlou1554
Refactor filter implementation
5c5a40f
xx01cyx
xx01cyx reviewed Nov 16, 2024
View reviewed changes
lanlou1554 and others added 23 commits November 16, 2024 23:19
@lanlou1554
Improve filter tests
d6e1825
@xx01cyx
refine test infra
082f0be
@xx01cyx
add test for cost model agg
e183f02
@xx01cyx
make all data types u64 instead of usize
0059141
@xx01cyx
merge main and resolve conflicts
303d73c
@xx01cyx
copy paste join cardinality calculation
ec0afa6
@xx01cyx
make join compile
6d50843
@xx01cyx
rename col -> attr
a4ff526
@xx01cyx
refactor join to not pass in logical props
0ba4132
@xx01cyx
make statistics f64 instead of u64
ab15f05
@xx01cyx
split join into multiple files
b682c73
@xx01cyx
reorganize join
5d73141
@xx01cyx
refine test infra
51f917d
@xx01cyx
add test infra for join
5197090
@xx01cyx
refine mock interface
68b2885
@xx01cyx
make CostModelStorageManagerImpl::get_attribute_info unimplemented
36b93b9
@xx01cyx
modify MemoExt interface
11a3a4e
@xx01cyx
rename AttrRefPred -> AttrIndexPred and revert back to initial design
8c4191f
@lanlou1554
Modify the tests of filter and agg
1569fc5
@xx01cyx
add join test
489ff48
@xx01cyx
pass group id to join and fix filter-related tests
be71afb
@xx01cyx
fix all join tests
624d040
@lanlou1554
Change filter controller name
f8a0e70
@lanlou1554
Copy link
Contributor Author

lanlou1554 commented Nov 20, 2024

Have been merge by other PRs #40 #41 #42

@lanlou1554 lanlou1554 closed this Nov 20, 2024
@lanlou1554 lanlou1554 deleted the cost-model-migration branch November 20, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xx01cyx xx01cyx xx01cyx left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lanlou1554 @xx01cyx